You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Summary of Changes

Hello, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request introduces several improvements across the codebase, focusing on code quality, accessibility, and user experience. It removes unused code, cleans up console logs, and replaces any types with more specific TypeScript types. Additionally, it enhances accessibility by using semantic HTML and improves the loading experience for NFTs.

Highlights

• Code Quality: This PR focuses on improving code quality by removing dead code, cleaning up console logs, and applying proper TypeScript types.
• Accessibility: The PR enhances accessibility by replacing clickable <div> elements with semantic <button> elements in several components.
• User Experience: Loading UX is improved by replacing plain text with skeleton loading UI in the NFT gallery.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

PR Review: refactor(ui): polish code quality across components

Overall this is a solid cleanup PR — -332/+53 lines with meaningful improvements across TypeScript safety, accessibility, and UX. Here's my detailed feedback:

✅ Strong improvements

• TypeScript any elimination in markdown-components.tsx, otp-box.tsx, apple-standalone.tsx, apple.tsx, my-file.tsx, input.tsx, preview4.tsx — all correct and well-typed
• catch (e: unknown) with proper narrowing in otp-box.tsx and my-file.tsx (e instanceof Error ? e.message : String(e)) is the right pattern
• Accessibility wins: confirm-dialog, nft-gallary-item, and avatar all replacing clickable <div> with <button> is correct
• Avatar conditional button render (props.onClick ? <button> : <div>) properly avoids adding interactive semantics when there's no handler
• NFT skeleton loading is a meaningful UX improvement over a plain text string
• Dead code removal (MetaMask, sign-in-with-apple) makes the intent clear

⚠️ Issues to address

1. confirm-dialog.tsx — nested interactive elements risk

The trigger wrapper changed from <div> to <button>, but children can contain arbitrary content including other <button> elements. Nesting <button> inside <button> is invalid HTML and causes browser parsing issues.

Consider using role="button" + tabIndex={0} + keyboard handler for a generic wrapper, or document that children must not contain interactive elements.

2. emoji-picker.tsx — silent no-op

// Before
onEmojiSelect={console.log}

// After
onEmojiSelect={() => {}}

Removing the console.log is right, but replacing it with a no-op means emoji selection silently does nothing. If this component is not yet wired up, add a // TODO comment or accept a callback prop. As-is, it looks like a working component that silently drops user input.

3. nft-gallary.tsx — non-standard Tailwind class

<div className="grid w-144 grid-cols-2 gap-4">

w-144 (576px) is not in the default Tailwind spacing scale (which stops at w-96). This would require either a custom tailwind.config.js entry or Tailwind v4's extended scale. Verify this renders correctly or use an explicit width like w-[576px] or a safe alternative.

4. background-upload-modal.tsx — mock function still in production code

const mockUploadBackground = useCallback(
  async (_imageUrl: string): Promise<void> => {
    return new Promise((resolve) => {
      setTimeout(() => {
        resolve()
      }, 1000)
    })
  },
  []
)

The function was renamed from imageUrl → _imageUrl and the console.log removed, but the mock (1s timeout, no actual upload) is still there. This is a pre-existing issue but worth flagging — if this runs in production it does nothing and silently "succeeds."

💡 Minor suggestions

• auth.metamask.tsx and metamask-standalone.tsx: These files now export components that return null. If MetaMask is permanently removed, consider removing the files entirely and removing their import sites — keeping stub files can mislead future contributors.

• nft-gallary-item.tsx button cursor: cursor-pointer was added to the button class, but buttons already have pointer cursor in most CSS resets. Minor redundancy, not harmful.

Summary

The nested button issue in confirm-dialog is the most important to resolve before merge. The rest are minor.

🤖 Reviewed with Claude Code